Skip to content

Configure application security in CI#38

Merged
shreddedbacon merged 7 commits intomainfrom
appsec-ci
May 20, 2025
Merged

Configure application security in CI#38
shreddedbacon merged 7 commits intomainfrom
appsec-ci

Conversation

@smlx
Copy link
Member

@smlx smlx commented Apr 15, 2025

Partially addresses: #32

Once this is merged the remaining config changes can be made to the repo and #32 closed.

@smlx smlx force-pushed the appsec-ci branch 2 times, most recently from 7174cd3 to 51d351c Compare May 16, 2025 03:55
@smlx smlx marked this pull request as ready for review May 16, 2025 04:04
@smlx smlx requested a review from shreddedbacon May 16, 2025 04:04
@smlx
Copy link
Member Author

smlx commented May 16, 2025

@shreddedbacon I added the tag-to-release workflow which is supposed to create the release on a tag push, so I can generate and attach an SBOM. Not actually tested though, so might need some extra work.

smlx added 5 commits May 16, 2025 13:18
This workflow will trigger when a tag is pushed and will create a Github
Release for the tag.

It will also attach an attested SBOM to the release.
@shreddedbacon
Copy link
Member

I'll fix up the lint-go stuff separately

@smlx
Copy link
Member Author

smlx commented May 16, 2025

Yeah I don't think the checks don't have to go green in this PR before merging.

@shreddedbacon
Copy link
Member

Is this ready for review now? Or are you still working through some changes?

@smlx
Copy link
Member Author

smlx commented May 19, 2025

Sorry it wasn't clear. Ready to review!

@shreddedbacon shreddedbacon merged commit 82230ea into main May 20, 2025
9 of 11 checks passed
@shreddedbacon shreddedbacon deleted the appsec-ci branch May 20, 2025 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants